Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.01%. Comparing base (d421b9e) to head (8f405ed).

@@            Coverage Diff             @@
##             main    #1560      +/-   ##
==========================================
- Coverage   52.04%   52.01%   -0.03%     
==========================================
  Files          36       36              
  Lines        4333     4333              
==========================================
- Hits         2255     2254       -1     
- Misses       2078     2079       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Error seems unrelated to the PR given all this PR does is extend the Extension class. The failing test:

    @pytest.mark.sklearn()
    @unittest.skipIf(
        Version(sklearn.__version__) == Version("0.19.1"),
        reason="Requires scikit-learn!=0.19.1, because target flow is from that version.",
    )
    @pytest.mark.production()
    def test_get_flow_with_reinstantiate_strict_with_wrong_version_raises_exception(self):
        self.use_production_server()
        flow = 8175
        expected = "Trying to deserialize a model with dependency sklearn==0.19.1 not satisfied."
        self.assertRaisesRegex(
            ValueError,
            expected,
            openml.flows.get_flow,
            flow_id=flow,
            reinstantiate=True,
            strict_version=True,
        )

Edit: The tests pass locally.

@jgyasu the problem in fact was related to this PR, if openml.extensions.extensions.clear() runs before test_get_flow_with_reinstantiate_strict_with_wrong_version_raises_exception, and import openml_sklearn is called before openml.extensions.extensions.clear() once, it will be cleared from the openml.extensions.extensions list, and further imports won't add it back given sys.modules will identify that it was imported previously and hence won't add the data it is supposed to to openml.extensions.extensions, hence causing failures!

Test failures discussed here